Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] test for ONEAPI_DEVICE_SELECTOR case sensitivity fix #1579

Merged
merged 2 commits into from
Feb 10, 2023

Conversation

cperkinsintel
Copy link

fix for ONEAPI_DEVICE_SELECTOR case sensitivity is here: intel/llvm#8238

// RUN: env ONEAPI_DEVICE_SELECTOR="OPENCL:*" %t.out
// RUN: env ONEAPI_DEVICE_SELECTOR="opencl:*" %t.out
// RUN: env ONEAPI_DEVICE_SELECTOR="*:GPU" %t.out
// RUN: env ONEAPI_DEVICE_SELECTOR="*:gpu" %t.out

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it make sense to have a test where it mixes upper and lowercase letters? I.e. something like OPeNcL or even an actual common spelling like OpenCL.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe. What if I removed all these cases for just one: "OpEnCl:GpU" ?

@cperkinsintel
Copy link
Author

/verify with intel/llvm#8238

@cperkinsintel
Copy link
Author

The "regular" test failures are expected. The Jenkins/llvm-test-suite run is the run against the PR with the new feature, and all tests are passing.

@againull againull self-requested a review February 10, 2023 22:55
@againull againull merged commit b600b13 into intel:intel Feb 10, 2023
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants